Skip to content

Comments

Fixed asset warnings mentioned in #4566#4567

Merged
shai-almog merged 2 commits intomasterfrom
ios-native-build-warning-fixes
Feb 23, 2026
Merged

Fixed asset warnings mentioned in #4566#4567
shai-almog merged 2 commits intomasterfrom
ios-native-build-warning-fixes

Conversation

@shai-almog
Copy link
Collaborator

  1. [asset-catalog] (41)

    Example:
    /Users/shai/dev/cn1/scripts/hellocodenameone/ios/target/hellocodenameone-ios-1.0-SNAPSHOT-ios-source/HelloCodenameOne-src/Images.xcassets:./AppIcon.appiconset/(null)[2d][Icon.png]: warning: The app icon set "AppIcon" has 3 unassigned children.
    Initial root-cause analysis:
    The generated Images.xcassets references icon/launch-image slots whose files are missing or unmapped in Contents.json.
    This is likely due to legacy launch-image/icon templates being emitted without matching assets for all size idioms.

1) [asset-catalog] (41)

    Example:
        /Users/shai/dev/cn1/scripts/hellocodenameone/ios/target/hellocodenameone-ios-1.0-SNAPSHOT-ios-source/HelloCodenameOne-src/Images.xcassets:./AppIcon.appiconset/(null)[2d][Icon.png]: warning: The app icon set "AppIcon" has 3 unassigned children.
    Initial root-cause analysis:
        The generated Images.xcassets references icon/launch-image slots whose files are missing or unmapped in Contents.json.
        This is likely due to legacy launch-image/icon templates being emitted without matching assets for all size idioms.
@github-actions
Copy link

github-actions bot commented Feb 23, 2026

✅ Continuous Quality Report

Test & Coverage

Static Analysis

Generated automatically by the PR CI workflow.

@shai-almog
Copy link
Collaborator Author

shai-almog commented Feb 23, 2026

Compared 31 screenshots: 31 matched.

Native Android coverage

  • 📊 Line coverage: 7.23% (3674/50829 lines covered) [HTML preview] (artifact android-coverage-report, jacocoAndroidReport/html/index.html)
    • Other counters: instruction 5.63% (18017/319865), branch 2.73% (848/31109), complexity 3.36% (1000/29734), method 5.94% (822/13848), class 9.73% (178/1829)
    • Lowest covered classes
      • kotlin.collections.kotlin.collections.ArraysKt___ArraysKt – 0.00% (0/6327 lines covered)
      • kotlin.collections.unsigned.kotlin.collections.unsigned.UArraysKt___UArraysKt – 0.00% (0/2384 lines covered)
      • org.jacoco.agent.rt.internal_b6258fc.asm.org.jacoco.agent.rt.internal_b6258fc.asm.ClassReader – 0.00% (0/1519 lines covered)
      • kotlin.collections.kotlin.collections.CollectionsKt___CollectionsKt – 0.00% (0/1148 lines covered)
      • org.jacoco.agent.rt.internal_b6258fc.asm.org.jacoco.agent.rt.internal_b6258fc.asm.MethodWriter – 0.00% (0/923 lines covered)
      • kotlin.sequences.kotlin.sequences.SequencesKt___SequencesKt – 0.00% (0/712 lines covered)
      • kotlin.text.kotlin.text.StringsKt___StringsKt – 0.00% (0/623 lines covered)
      • org.jacoco.agent.rt.internal_b6258fc.asm.org.jacoco.agent.rt.internal_b6258fc.asm.Frame – 0.00% (0/564 lines covered)
      • kotlin.collections.kotlin.collections.ArraysKt___ArraysJvmKt – 0.00% (0/495 lines covered)
      • kotlinx.coroutines.kotlinx.coroutines.JobSupport – 0.00% (0/423 lines covered)

✅ Native Android screenshot tests passed.

Native Android coverage

  • 📊 Line coverage: 7.23% (3674/50829 lines covered) [HTML preview] (artifact android-coverage-report, jacocoAndroidReport/html/index.html)
    • Other counters: instruction 5.63% (18017/319865), branch 2.73% (848/31109), complexity 3.36% (1000/29734), method 5.94% (822/13848), class 9.73% (178/1829)
    • Lowest covered classes
      • kotlin.collections.kotlin.collections.ArraysKt___ArraysKt – 0.00% (0/6327 lines covered)
      • kotlin.collections.unsigned.kotlin.collections.unsigned.UArraysKt___UArraysKt – 0.00% (0/2384 lines covered)
      • org.jacoco.agent.rt.internal_b6258fc.asm.org.jacoco.agent.rt.internal_b6258fc.asm.ClassReader – 0.00% (0/1519 lines covered)
      • kotlin.collections.kotlin.collections.CollectionsKt___CollectionsKt – 0.00% (0/1148 lines covered)
      • org.jacoco.agent.rt.internal_b6258fc.asm.org.jacoco.agent.rt.internal_b6258fc.asm.MethodWriter – 0.00% (0/923 lines covered)
      • kotlin.sequences.kotlin.sequences.SequencesKt___SequencesKt – 0.00% (0/712 lines covered)
      • kotlin.text.kotlin.text.StringsKt___StringsKt – 0.00% (0/623 lines covered)
      • org.jacoco.agent.rt.internal_b6258fc.asm.org.jacoco.agent.rt.internal_b6258fc.asm.Frame – 0.00% (0/564 lines covered)
      • kotlin.collections.kotlin.collections.ArraysKt___ArraysJvmKt – 0.00% (0/495 lines covered)
      • kotlinx.coroutines.kotlinx.coroutines.JobSupport – 0.00% (0/423 lines covered)

@shai-almog
Copy link
Collaborator Author

shai-almog commented Feb 23, 2026

Compared 31 screenshots: 31 matched.
✅ Native iOS screenshot tests passed.

Benchmark Results

  • VM Translation Time: 344 seconds
  • Compilation Time: 304 seconds

Detailed Performance Metrics

Metric Duration
Build Time Statistics
Setup & Unzip 31175 ms
Extract Extensions 22 ms
Google Services Setup 2 ms
Scan Classes 622 ms
Extract Libs 1107 ms
Inject Build Hints 60 ms
Generate Unit Tests 3 ms
Generate Stubs 977 ms
Compile Stubs 2286 ms
Generate Icons 1076 ms
Prepare ParparVM 296 ms
ParparVM Execution 211257 ms
Post-VM Setup 223 ms
CocoaPods 5328 ms
Finalize 50 ms
Total Time 254484 msMaven Overhead : 90000 ms
CocoaPods Install (Script) 1000 ms
Simulator Boot 76000 ms
Simulator Boot (Run) 3000 ms
App Install 16000 ms
App Launch 13000 ms
Test Execution 175000 ms

…ation

[-Wdeprecated-declarations] (10)

    Example:
        cn1/scripts/hellocodenameone/ios/target/hellocodenameone-ios-1.0-SNAPSHOT-ios-source/HelloCodenameOne-src/com_codenameone_examples_hellocodenameone_LocalNotificationNativeImpl.m:12:35: warning: 'scheduledLocalNotifications' is deprecated: first deprecated in iOS 10.0 - Use UserNotifications Framework's -[UNUserNotificationCenter getPendingNotificationRequestsWithCompletionHandler:] [-Wdeprecated-declarations]
    Initial root-cause analysis:
        App-native implementation still uses UILocalNotification era APIs.
        Migration to UserNotifications (UNUserNotificationCenter, UNNotificationRequest) would remove these warnings.
@shai-almog shai-almog merged commit b957b39 into master Feb 23, 2026
14 checks passed
@codenameone codenameone deleted a comment from TonGalaxy Feb 24, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant